Skip to content

Conversation

@cigaly
Copy link
Contributor

@cigaly cigaly commented Dec 2, 2024

Jira issue HHH-18858

When array property is annotated with one of @ManyToMany, @OneToMany, or @ElementCollection, field in generated meta model class should be of type jakarta.persistence.metamodel.ListAttribute, currently is jakarta.persistence.metamodel.SingularAttribute.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license
and can be relicensed under the terms of the LGPL v2.1 license in the future at the maintainers' discretion.
For more information on licensing, please check here.


https://hibernate.atlassian.net/browse/HHH-18868

@gavinking
Copy link
Member

gavinking commented Dec 4, 2024

@cigaly Is this actually for HHH-18868?

@cigaly
Copy link
Contributor Author

cigaly commented Dec 4, 2024

No, for HHH-18868 is PR #9353

This one is creating ListAttribute field for array properties annotated as OneToMany, ManyToOne, or ElementCollection. If I not forgot something, other array properties should be treated as SingleAttribute

@gavinking
Copy link
Member

gavinking commented Dec 4, 2024

Oh, sorry, it's me who got mixed up. Sorry.

Copy link
Member

@gavinking gavinking left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, can I merge it if the tests succeed?

@cigaly
Copy link
Contributor Author

cigaly commented Dec 4, 2024

Yes, of course. [I'll remove draft status when tests are finished]

@cigaly cigaly marked this pull request as ready for review December 4, 2024 12:43
@gavinking gavinking merged commit a38a300 into hibernate:main Dec 4, 2024
22 of 23 checks passed
@gavinking
Copy link
Member

Thanks so much @cigaly!

By the way, I was trying to contact you on Zulip....

@cigaly
Copy link
Contributor Author

cigaly commented Dec 4, 2024

... and you did not found me 'cause I have not been registered. I guess that now you can.

@cigaly
Copy link
Contributor Author

cigaly commented Dec 5, 2024

@cigaly Is this actually for HHH-18868?

Sorry, my mistake. I've just seen that comments in commits are mentioning wrong issue :-(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants